Skip to content

BUG: resample Grouper in a list grouping on a column with NaT throws an error #22470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 28, 2018

Conversation

discort
Copy link
Contributor

@discort discort commented Aug 22, 2018

@codecov
Copy link

codecov bot commented Aug 23, 2018

Codecov Report

Merging #22470 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #22470   +/-   ##
=======================================
  Coverage   92.04%   92.04%           
=======================================
  Files         169      169           
  Lines       50769    50769           
=======================================
  Hits        46731    46731           
  Misses       4038     4038
Flag Coverage Δ
#multiple 90.45% <ø> (ø) ⬆️
#single 42.23% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 982f309...34dda28. Read the comment docs.

@@ -3175,6 +3175,15 @@ def test_median_duplicate_columns(self):
expected.columns = result.columns
assert_frame_equal(result, expected)

def test_resample_list_grouper_with_nat(self):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you test the case w/o a list as well

move test to pandas/tests/groupby/test_grouping.py

@jreback jreback added Testing pandas testing functions or related to the test suite Groupby Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate labels Aug 23, 2018
@jreback jreback added this to the 0.24.0 milestone Aug 23, 2018
@discort discort force-pushed the list_grouper_resample branch from c52a98d to 34dda28 Compare August 23, 2018 10:51
@discort
Copy link
Contributor Author

discort commented Aug 27, 2018

@jreback

@jreback jreback merged commit 95bc7d1 into pandas-dev:master Aug 28, 2018
@jreback
Copy link
Contributor

jreback commented Aug 28, 2018

thanks!

Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this pull request Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Groupby Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: resample Grouper in a list grouping on a column with NaT throws an error
2 participants